-
Notifications
You must be signed in to change notification settings - Fork 71
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bugfix/issue#266 cmdjlink permission denied #295
Bugfix/issue#266 cmdjlink permission denied #295
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks Ok
Looks good to me. How did you test it so far? |
Yep, looks good 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we before merge test it in linux and macos?
@ederjc @jaenrig-ifx |
What is the deal with M2 macbooks? |
- fix erase parameter
f73ecc0
to
593cd19
Compare
I'm just unfamiliar with macbook myself 😿 |
…nk_permission_denied Bugfix/issue#266 cmdjlink permission denied Flashing script update: verify macOS, verbose log
By creating this pull request you agree to the terms in CONTRIBUTING.md.
https://github.com/Infineon/.github/blob/master/CONTRIBUTING.md
--- DO NOT DELETE ANYTHING ABOVE THIS LINE ---
CONTRIBUTING.md also tells you what to expect in the PR process.
Description
The download path creates permission problems if the Arduino IDE is installed for all users of Windows.
Related Issue
#266
Context
Store the intermediate jlink files in temporary folder to solve the permission problem.
This shoud work for all OS - test needed:
Windows
Linux
MacOS
There were some problem with macOS:
so I changed "python" to "python3". for windows I kept "python"